Skip to content

fix(google-maps): missing types for <ScriptGoogleMaps/>#560

Merged
harlan-zw merged 1 commit intonuxt:mainfrom
BobbieGoede:fix/script-google-maps-component-types
Dec 13, 2025
Merged

fix(google-maps): missing types for <ScriptGoogleMaps/>#560
harlan-zw merged 1 commit intonuxt:mainfrom
BobbieGoede:fix/script-google-maps-component-types

Conversation

@BobbieGoede
Copy link
Member

🔗 Linked issue

❓ Type of change

  • 📖 Documentation (updates to the documentation or readme)
  • 🐞 Bug fix (a non-breaking change that fixes an issue)
  • 👌 Enhancement (improving an existing functionality)
  • ✨ New feature (a non-breaking change that adds functionality)
  • 🧹 Chore (updates to the build process or auxiliary tools and libraries)
  • ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

The generated types for the <ScriptGoogleMaps/> is empty, see the dist/runtime/components/GoogleMaps/ScriptGoogleMaps.d.vue.ts file.

You can see the following error while building this module:

src/runtime/components/GoogleMaps/ScriptGoogleMaps.vue(22,25): error TS4025: Exported variable '__VLS_export' has or is using private name 'PlaceholderOptions'.

Based on this error I exported the PlaceholderOptions in the component, which resolves the error and type generation.

This behavior is not present in the playground setup, since it is directly importing source code instead of using a stub/build/package. There may be other functionality that work in the playground that differ in actual projects, this issue in particular is an outlier in that it logs an error during build.

@vercel
Copy link
Contributor

vercel bot commented Dec 12, 2025

@BobbieGoede is attempting to deploy a commit to the Nuxt Team on Vercel.

A member of the Team first needs to authorize it.

@pkg-pr-new
Copy link

pkg-pr-new bot commented Dec 12, 2025

Open in StackBlitz

npm i https://pkg.pr.new/nuxt/scripts/@nuxt/scripts@560

commit: 00abee9

Copy link
Collaborator

@harlan-zw harlan-zw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@harlan-zw harlan-zw merged commit 5021ac2 into nuxt:main Dec 13, 2025
4 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants